Skip to content
This repository was archived by the owner on Oct 4, 2021. It is now read-only.

Conversation

@monojenkins
Copy link
Contributor

@monojenkins monojenkins commented Nov 14, 2019

There are some cases where, it seems, we don't evaluate conditions as we should,
resulting in duplicated items being added to the project and, thus, also shown
in the project tree (see dotnet/aspnetcore#17088).

Also, by just adding a , duplicated files show also.

So, due to this limitation on the project system, work around it on the project
tree, by checking for duplicates based on the files' paths.

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1005277

Backport of #9309.

/cc @rodrmoya

@rodrmoya rodrmoya self-assigned this Nov 14, 2019
@rodrmoya rodrmoya requested a review from Therzok November 14, 2019 19:31
@rodrmoya rodrmoya added this to the 8.4 milestone Nov 14, 2019
There are some cases where, it seems, we don't evaluate conditions as we should,
resulting in duplicated items being added to the project and, thus, also shown
in the project tree (see dotnet/aspnetcore#17088).

Also, by just adding a <None Include="*"/>, duplicated files show also.

So, due to this limitation on the project system, work around it on the project
tree, by checking for duplicates based on the files' paths.

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1005277
@monojenkins monojenkins force-pushed the backport-pr-9309-to-release-8.4 branch from 31adb56 to 90f09d7 Compare November 14, 2019 19:55
@slluis
Copy link
Member

slluis commented Nov 14, 2019

I'm not sure this is correct, the issue should be fixed in the project system. I'd like @mrward to take a look.

@rodrmoya rodrmoya closed this Dec 2, 2019
@rodrmoya rodrmoya deleted the backport-pr-9309-to-release-8.4 branch December 2, 2019 19:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants